Skip to content

feat!: Remove setupEnzyme helper and unmark RadialProgress as deprecated#3337

Merged
aresnik11 merged 4 commits intomainfrom
ajr-remove-setup-enzyme
May 4, 2026
Merged

feat!: Remove setupEnzyme helper and unmark RadialProgress as deprecated#3337
aresnik11 merged 4 commits intomainfrom
ajr-remove-setup-enzyme

Conversation

@aresnik11
Copy link
Copy Markdown
Contributor

@aresnik11 aresnik11 commented May 1, 2026

Overview

Removes setupEnzyme helper from gamut-tests. Removes deprecated comment from RadialProgress as it is not truly deprecated.

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: GMT-1626, GMT-1627
  • Version plan added/updated (or not needed)
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

  1. Confirm all checks pass
  2. I've confirmed none of our repos use setupEnzyme
  3. Finish and do a celebratory dance

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 1, 2026

View your CI Pipeline Execution ↗ for commit 744b16f


☁️ Nx Cloud last updated this comment at 2026-05-04 17:57:45 UTC

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 1, 2026

View your CI Pipeline Execution ↗ for commit 94a0e4b


☁️ Nx Cloud last updated this comment at 2026-05-01 17:58:55 UTC

@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.92%. Comparing base (80719f3) to head (744b16f).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3337      +/-   ##
==========================================
- Coverage   90.40%   89.92%   -0.49%     
==========================================
  Files         405      278     -127     
  Lines        6649     5736     -913     
  Branches     2156     1897     -259     
==========================================
- Hits         6011     5158     -853     
+ Misses        630      570      -60     
  Partials        8        8              
Flag Coverage Δ
main ?
pull-request 89.92% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aresnik11 aresnik11 changed the title remove setupenzyme feat!: Remove setupEnzyme helper and unmark RadialProgress as deprecated May 4, 2026
@aresnik11 aresnik11 marked this pull request as ready for review May 4, 2026 15:41
@aresnik11 aresnik11 requested a review from a team as a code owner May 4, 2026 15:41
@aresnik11 aresnik11 requested a review from dreamwasp May 4, 2026 15:41
@codecademydev
Copy link
Copy Markdown
Collaborator

📬 Published Alpha Packages:

Package Version npm Diff
@codecademy/gamut 68.4.2-alpha.63a45b.0 npm diff
@codecademy/gamut-icons 9.57.4-alpha.63a45b.0 npm diff
@codecademy/gamut-illustrations 0.58.11-alpha.63a45b.0 npm diff
@codecademy/gamut-kit 0.6.596-alpha.63a45b.0 npm diff
@codecademy/gamut-patterns 0.10.30-alpha.63a45b.0 npm diff
@codecademy/gamut-styles 17.14.1-alpha.63a45b.0 npm diff
@codecademy/gamut-tests 5.3.5-alpha.63a45b.0 npm diff
@codecademy/variance 0.26.2-alpha.63a45b.0 npm diff
eslint-plugin-gamut 2.4.4-alpha.63a45b.0 npm diff

@aresnik11 aresnik11 merged commit 7ba4334 into main May 4, 2026
14 checks passed
@aresnik11 aresnik11 deleted the ajr-remove-setup-enzyme branch May 4, 2026 17:58
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants